Fixed GtkCellLayout buildable implementation to set cell properties explicitly set...
authorTristan Van Berkom <tristan.van.berkom@gmail.com>
Wed, 2 Feb 2011 06:20:42 +0000 (15:20 +0900)
committerTristan Van Berkom <tristan.van.berkom@gmail.com>
Wed, 2 Feb 2011 06:22:31 +0000 (15:22 +0900)
gtk/gtkcelllayout.c

index d0ff6caacce10ac8c6afb5d789ea969655ede7ff..3fd59ade6ba00344a67543083d7a05d35aa0e34e 100644 (file)
@@ -807,37 +807,35 @@ cell_packing_end_element (GMarkupParseContext *context,
   CellPackingSubParserData *parser_data = (CellPackingSubParserData*)user_data;
   GtkCellArea *area;
 
-  /* Append the translated strings */
-  if (parser_data->string->len)
-    {
-      area = gtk_cell_layout_get_area (parser_data->cell_layout);
+  area = gtk_cell_layout_get_area (parser_data->cell_layout);
 
-      if (area)
+  if (area)
+    {
+      /* translate the string */
+      if (parser_data->string->len && parser_data->translatable)
        {
-         if (parser_data->translatable)
-           {
-             gchar *translated;
-             const gchar* domain;
-
-             domain = gtk_builder_get_translation_domain (parser_data->builder);
-
-             translated = _gtk_builder_parser_translate (domain,
-                                                         parser_data->context,
-                                                         parser_data->string->str);
-             g_string_set_size (parser_data->string, 0);
-             g_string_append (parser_data->string, translated);
-           }
-
-         gtk_cell_layout_buildable_set_cell_property (area, 
-                                                      parser_data->builder,
-                                                      parser_data->renderer,
-                                                      parser_data->cell_prop_name,
-                                                      parser_data->string->str);
+         gchar *translated;
+         const gchar* domain;
+
+         domain = gtk_builder_get_translation_domain (parser_data->builder);
+
+         translated = _gtk_builder_parser_translate (domain,
+                                                     parser_data->context,
+                                                     parser_data->string->str);
+         g_string_set_size (parser_data->string, 0);
+         g_string_append (parser_data->string, translated);
        }
-      else
-       g_warning ("%s does not have an internal GtkCellArea class and cannot apply child cell properties",
-                  g_type_name (G_OBJECT_TYPE (parser_data->cell_layout)));
+
+      if (parser_data->cell_prop_name)
+       gtk_cell_layout_buildable_set_cell_property (area, 
+                                                    parser_data->builder,
+                                                    parser_data->renderer,
+                                                    parser_data->cell_prop_name,
+                                                    parser_data->string->str);
     }
+  else
+    g_warning ("%s does not have an internal GtkCellArea class and cannot apply child cell properties",
+              g_type_name (G_OBJECT_TYPE (parser_data->cell_layout)));
 
   g_string_set_size (parser_data->string, 0);
   g_free (parser_data->cell_prop_name);